Skip to content

Conversation

makai410
Copy link
Contributor

@makai410 makai410 commented Aug 18, 2025

  • devtool: A helper for maintaining rustc_public. The main logic for commands like build is implemented here. This PR implements ./x build, ./x test [--bless], ./x fmt [--check], ./x githook install/uninstall and ./x clean. Windows is not supported yet.
  • This PR turns the compiletest crate into a single file, making it possilbe to run test suites using cargo test.

@celinval
Copy link
Contributor

Amazing! On the file structure, since this repo is also meant to store files related to the project, can I suggest that we create a rustc_public folder with the code relevant for the crate. Or would that mess up with the josh-sync?

@celinval
Copy link
Contributor

I was also wondering if you could break down the PR a bit more. I think there are changes here that are ready to be merged independently

@makai410
Copy link
Contributor Author

Amazing! On the file structure, since this repo is also meant to store files related to the project, can I suggest that we create a rustc_public folder with the code relevant for the crate. Or would that mess up with the josh-sync?

It seems feasible by writing a filter to the josh, though I haven't tried it yet. IMO it's OK to keep our rustc_public code in the src/ folder, and miri does the same.

I was also wondering if you could break down the PR a bit more. I think there are changes here that are ready to be merged independently

Yea makes sense. I'll split it up.

@makai410 makai410 marked this pull request as ready for review August 25, 2025 12:51
@oli-obk
Copy link
Contributor

oli-obk commented Aug 26, 2025

CI wants some rustfmting

env:
TOOLS_BIN: "/tmp/smir/bin"
# Note that the downloaded version is dated 2025-08-09.
toolchain: nightly-2025-08-10
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer a toolchain toml file, as we'll update it frequently and that's easier to automate than editing a github workflow

Fine for now tho if you want to land this first and tune later

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah makes sense. I'd like to update it later together with the automation pr

@oli-obk
Copy link
Contributor

oli-obk commented Aug 26, 2025

This changes the rustc_public crate. Can you check that it still works inside the rust repo if synced back?

@makai410
Copy link
Contributor Author

The rustc compilation fails directly since we have a really strict msrv in our build.rs. Can we use the :exclude filter to avoid syncing those changes back?

@makai410
Copy link
Contributor Author

wait, is it possible to keep the files for the crates.io version in a separate directory? We can specify which Cargo.toml to use when building, and Cargo.toml itself can define the lib and build files.

@oli-obk
Copy link
Contributor

oli-obk commented Aug 29, 2025

wait, is it possible to keep the files for the crates.io version in a separate directory? We can specify which Cargo.toml to use when building, and Cargo.toml itself can define the lib and build files.

Yea. I think the plan was to overwrite the sources of the crates.io copy every time we make a breaking change?

@makai410
Copy link
Contributor Author

Yes the rustc version would be the base of a new major release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants